Skip to content

ci: add sync-translations workflow for zh-cn#723

Closed
mmcky wants to merge 2 commits into
mainfrom
add/sync-translations-zh-cn
Closed

ci: add sync-translations workflow for zh-cn#723
mmcky wants to merge 2 commits into
mainfrom
add/sync-translations-zh-cn

Conversation

@mmcky

@mmcky mmcky commented Apr 9, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Adds a GitHub Actions workflow that automatically translates changed lectures into Simplified Chinese on merged PRs, targeting QuantEcon/lecture-intro.zh-cn via QuantEcon/action-translation@v0.14.1.

Details

  • Trigger: On merged PRs that modify lectures/**/*.md or lectures/_toc.yml
  • Re-trigger: Comment \translate-resync on a merged PR
  • Target repo: QuantEcon/lecture-intro.zh-cn
  • Action: QuantEcon/action-translation@v0.14.1

Based on the same pattern used in lecture-python-programming.

Prerequisites

Ensure the following secrets are configured in this repository:

  • ANTHROPIC_API_KEY
  • QUANTECON_SERVICES_PAT (with write access to QuantEcon/lecture-intro.zh-cn)

TODO

  • setup review action-translation on zh-cn repo

@mmcky
ci: add sync-translations workflow for zh-cn
4dc07cb 
Adds a GitHub Actions workflow that automatically translates changed
lectures into Simplified Chinese on merged PRs, targeting
QuantEcon/lecture-intro.zh-cn via QuantEcon/action-translation@v0.14.1.

Supports re-triggering via \translate-resync comment on merged PRs.
Copilot AI review requested due to automatic review settings April 9, 2026 05:29
@netlify

netlify Bot commented Apr 9, 2026 •
edited
Loading

Copy link
Copy Markdown

✅ Deploy Preview for taupe-gaufre-c4e660 ready!

Name Link
🔨 Latest commit 7869db0
🔍 Latest deploy log https://app.netlify.com/projects/taupe-gaufre-c4e660/deploys/69d747ca25c1cc000843e804
😎 Deploy Preview https://deploy-preview-723--taupe-gaufre-c4e660.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

Copilot AI reviewed Apr 9, 2026
View reviewed changes

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds an automated GitHub Actions workflow to sync/translate updated lecture content into the Simplified Chinese target repository (QuantEcon/lecture-intro.zh-cn) when relevant PRs are merged, with an optional comment-based re-sync trigger.

Changes:

  • Introduces a new workflow triggered on merged PRs affecting lectures/**/*.md or lectures/_toc.yml.
  • Adds an issue_comment re-trigger mechanism via \translate-resync.
  • Runs QuantEcon/action-translation@v0.14.1 in sync mode targeting zh-cn.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread .github/workflows/sync-translations-zh-cn.yml Outdated
Comment thread .github/workflows/sync-translations-zh-cn.yml Outdated
@mmcky mmcky mentioned this pull request Apr 9, 2026
Merged
@github-actions

github-actions Bot commented Apr 9, 2026 •
edited
Loading

Copy link
Copy Markdown

🚀 Deployed on https://69d749c3dfc053cf9d3c9d72--taupe-gaufre-c4e660.netlify.app

@github-actions
github-actions Bot temporarily deployed to pull request April 9, 2026 05:37 Inactive
@github-actions
github-actions Bot temporarily deployed to pull request April 9, 2026 05:38 Inactive
@mmcky
fix: address Copilot review feedback on translation workflow
7869db0 
- Gate issue_comment re-sync to OWNER/MEMBER/COLLABORATOR only
- Verify comment is on a PR (not a plain issue)
- Resolve the merged PR's commit SHA for correct checkout on re-sync
- Verify the PR is actually merged before proceeding
@github-actions
github-actions Bot temporarily deployed to pull request April 9, 2026 06:40 Inactive
@github-actions
github-actions Bot temporarily deployed to pull request April 9, 2026 06:40 Inactive
@mmcky

mmcky commented Jul 18, 2026

Copy link
Copy Markdown
Contributor Author

Superseded by #795, which landed 2026-07-18 with the current fleet pattern (action-translation@v0, checkout@v7, QUANTECON_SERVICES_PAT) as part of the Phase 0 Track A wiring (#782). The sync is live and verified end-to-end — see QuantEcon/lecture-intro.zh-cn#267 for the first synced PR. Thanks for staging this in April; the workflow that merged is its direct descendant.

@mmcky mmcky closed this Jul 18, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

Copilot code review Copilot Copilot left review comments

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@mmcky